Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: [IOCOM-2100] Common FIMS start API #6740

Open
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

Vangaorth
Copy link
Contributor

@Vangaorth Vangaorth commented Feb 20, 2025

⚠️ This PR depends on 939 ⚠️
⚠️ This PR depends on 468 ⚠️

Short description

This PR centralise the starting of the FIMS authentication flow.

List of changes proposed in this pull request

There are now three hooks that allow to properly start the FIMS flow from a react-native component

  • useFIMSFromServiceId: when you know the service Id ahead of time and the service details are already loaded into memory (e.g, from message and service details)
  • useFIMSAuthenticationFlow: when you do not know the service Id ahead of time but will have it upon authentication flow invokation (e.g., a bonus of IdPay)
  • useFIMSRemoteServiceConfiguration: when the service data are stored in the CDN and you know the configuration id associated with such configuration (e.g., from the wallet driving license). Be aware that calling this hook with an invalid configuration Id will not start the authentication flow later on

All these hooks return an authentication callback that starts the FIMS authentication flow upon invocation.

How to test

Using the io-dev-api-server, check that all existing FIMS authentication flows are working as expected.

Copy link
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

Copy link
Contributor

github-actions bot commented Feb 20, 2025

Jira Pull Request Link

This Pull Request refers to Jira issues:

@Vangaorth Vangaorth marked this pull request as ready for review February 26, 2025 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant